Skip to content

Add form validation to login - #121

Merged
anoopcodehack merged 2 commits into
anoopcodehack:mainfrom
rimshashafique99:fix/login-form-validation
Jul 31, 2026
Merged

Add form validation to login#121
anoopcodehack merged 2 commits into
anoopcodehack:mainfrom
rimshashafique99:fix/login-form-validation

Conversation

@rimshashafique99

Copy link
Copy Markdown

Fixes #102

What changed:
Added validation to the login form before submit:

  • Checks email contains "@"
  • Checks password is at least 6 characters
  • Checks name is required when registering

Testing:
Manually tested empty fields, invalid email, and short password β€” error messages display correctly using the existing error UI.

@anoopcodehack
anoopcodehack merged commit f19efef into anoopcodehack:main Jul 31, 2026
@anoopcodehack

Copy link
Copy Markdown
Owner

Hey, @rimshashafique99 your PR has been merged! You're officially a DevBoard contributor now!

If you enjoyed contributing and found the project useful, consider leaving a ⭐ on the repo β€” it really helps more developers discover the project and keeps the community growing!

Also feel free to pick up another issue anytime β€” always happy to have you here! πŸ™Œ

@rimshashafique99

Copy link
Copy Markdown
Author

Thank you! This was my first open source contribution really appreciated the clear issue description and quick review. Will definitely pick up another issue soon πŸ™Œ

@rimshashafique99
rimshashafique99 deleted the fix/login-form-validation branch July 31, 2026 14:57
@anoopcodehack

Copy link
Copy Markdown
Owner

Hey, @rimshashafique99 your PR has been merged! You're officially a DevBoard contributor now!

If you enjoyed contributing and found the project useful, consider leaving a ⭐ on the repo β€” it really helps more developers discover the project and keeps the community growing!

Also feel free to pick up another issue anytime β€” always happy to have you here! πŸ™Œ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Login form has zero validationπŸ”

3 participants